Skip to content
This repository was archived by the owner on Nov 7, 2025. It is now read-only.

Conversation

@rhrazdil
Copy link
Contributor

@rhrazdil rhrazdil commented Aug 8, 2025

[Why]
We're in a process of migrating data from elasticsearch to clickhouse. We have a alerting service reading the data from elasticsearch (https://github.com/jertel/elastalert2), with no clear substitution offering feature partiy for Clickhouse. This is where quesma seems like a perfect fit.
One of our alerts relies on fuzzy search (~), so I decided to give it a try implementing.
I'd be happy for any feedback.

PR introduces implementation of elasticsearch fuzzy search for clickhouse database.
I decided to use damerauLevenshteinDistance clickhouse function, as its functionality is the closest to elasticsearch ~ operator, based on my direct comparison.

I did basic E2E test of the implemented functionality in my local setup with clickhouse-server 24.5.

Using damerauLevenshteinDistance clickhouse function as it's the closest
alternative to elasticsearch '~' operator.

Signed-off-by: Radim Hrazdil <[email protected]>
@rhrazdil rhrazdil requested a review from a team as a code owner August 8, 2025 06:43
@CLAassistant
Copy link

CLAassistant commented Aug 8, 2025

CLA assistant check
All committers have signed the CLA.

@trzysiek
Copy link
Member

trzysiek commented Aug 8, 2025

/run-it

@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 8, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
10.8% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@trzysiek
Copy link
Member

trzysiek commented Aug 8, 2025

Code LGTM, also tested it a bit manually. Thanks for the contribution!

@mieciu
Copy link
Member

mieciu commented Aug 11, 2025

/run-it

Copy link
Member

@mieciu mieciu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thank you so much for your contribution!

@mieciu mieciu added this pull request to the merge queue Aug 11, 2025
Merged via the queue into QuesmaOrg:main with commit 4438739 Aug 11, 2025
7 of 11 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants